From 2a75eac8cda756d414f6b23d9606fcb3ac376861 Mon Sep 17 00:00:00 2001 From: Olaf Hering Date: Thu, 26 May 2011 12:34:44 +0100 Subject: [PATCH] xentrace: reduce trace buffer size to something mfn_offset can reach The start of the array which holds the list of mfns for each cpus tracebuffer is stored in an unsigned short. This limits the total amount of pages for each cpu as the number of active cpus increases. Update the math in calculate_tbuf_size() to apply also this rule to the max number of trace pages. Without this change the index can overflow. Signed-off-by: Olaf Hering Acked-by: George Dunlap --- xen/common/trace.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/xen/common/trace.c b/xen/common/trace.c index b03ce5c38a..111deed3a5 100644 --- a/xen/common/trace.c +++ b/xen/common/trace.c @@ -112,11 +112,14 @@ static int calculate_tbuf_size(unsigned int pages, uint32_t t_info_first_offset) typeof(dummy_size.prod) max_size; struct t_info dummy_pages; typeof(dummy_pages.tbuf_size) max_pages; + typeof(dummy_pages.mfn_offset[0]) max_mfn_offset; + unsigned int max_cpus = num_online_cpus(); unsigned int t_info_words; /* force maximum value for an unsigned type */ max_size = -1; max_pages = -1; + max_mfn_offset = -1; /* max size holds up to n pages */ max_size /= PAGE_SIZE; @@ -124,6 +127,18 @@ static int calculate_tbuf_size(unsigned int pages, uint32_t t_info_first_offset) if ( max_size < max_pages ) max_pages = max_size; + /* + * max mfn_offset holds up to n pages per cpu + * The array of mfns for the highest cpu can start at the maximum value + * mfn_offset can hold. So reduce the number of cpus and also the mfn_offset. + */ + max_mfn_offset -= t_info_first_offset - 1; + max_cpus--; + if ( max_cpus ) + max_mfn_offset /= max_cpus; + if ( max_mfn_offset < max_pages ) + max_pages = max_mfn_offset; + if ( pages > max_pages ) { printk(XENLOG_INFO "xentrace: requested number of %u pages " -- 2.30.2